Skip to content

Feat/detail panel filepath view in condition - #63304

Open
redsteadz wants to merge 2 commits into
nextcloud:masterfrom
redsteadz:feat/detail-panel-filepath-view-in-condition
Open

Feat/detail panel filepath view in condition#63304
redsteadz wants to merge 2 commits into
nextcloud:masterfrom
redsteadz:feat/detail-panel-filepath-view-in-condition

Conversation

@redsteadz

@redsteadz redsteadz commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • the context menu has an option 'View In Folder' menu item available in the Files app
  • Added a file path below the name in files detail page

TODO:

  • Need to add before and after changes for the frontend.

Before:

  • File Details:
image
  • File View In Folder context item:
image

After:

  • File Details
image - File View In Folder context item image

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI
    The tests written were made using the help of GPT-5.6 Sol

@redsteadz
redsteadz requested review from a team and skjnldsv as code owners August 16, 2026 19:00
@redsteadz
redsteadz requested review from kristian-zendato and susnux and removed request for a team August 16, 2026 19:00
Assisted-by: OpenCode:gpt-5.6-sol
Signed-off-by: redsteadz <harcolemas@gmail.com>
@redsteadz
redsteadz force-pushed the feat/detail-panel-filepath-view-in-condition branch from 0567f26 to ed82de3 Compare August 16, 2026 19:24
}

// Only works outside of the main files view
if (view.id === 'files') {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when is this needed? Meaning when is this action ever useful in the files view?
As the files view is already the action target view this would end up in?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When doing search this action never appears, even though the file is within some xyz folder, and instead of just the file. I want to go to the actual folder. So instead of just disabling by view.id files, we can check if we are in the same folder as actual file, then no need to show this.

Comment on lines +25 to +26
<div class="filesSidebarSubname__path">
{{ node.path }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when the filepath is visible from the current path (e.g. files view or folder tree) then this is quite confusing to people. IMHO this should be move to its own section to make it a bit cleaner.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okk, fair point, will update with a different section and a label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

File search: Information Tab should contain file path

2 participants